Skip to content

ref: Remove PlanTier prop from subscription#117881

Merged
noahsmartin merged 2 commits into
masterfrom
removePlanTierProp
Jun 16, 2026
Merged

ref: Remove PlanTier prop from subscription#117881
noahsmartin merged 2 commits into
masterfrom
removePlanTierProp

Conversation

@noahsmartin

Copy link
Copy Markdown
Contributor

This is finally not used! Only a handful of places that sent it as an analytics tag, most also tracked the plan id itself which has more information than the tier. Removed these analytics use cases and completely deleted the type definition.

@noahsmartin noahsmartin requested review from a team as code owners June 16, 2026 22:29
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 16, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 51ce5c2. Configure here.

Comment thread static/gsAdmin/components/customerStatus.tsx
@noahsmartin noahsmartin merged commit 037afca into master Jun 16, 2026
78 checks passed
@noahsmartin noahsmartin deleted the removePlanTierProp branch June 16, 2026 22:48
billyvg pushed a commit that referenced this pull request Jun 17, 2026
This is finally not used! Only a handful of places that sent it as an
analytics tag, most also tracked the plan id itself which has more
information than the tier. Removed these analytics use cases and
completely deleted the type definition.
sehr-m pushed a commit that referenced this pull request Jun 23, 2026
This is finally not used! Only a handful of places that sent it as an
analytics tag, most also tracked the plan id itself which has more
information than the tier. Removed these analytics use cases and
completely deleted the type definition.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants